Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: setStoreV2 with StoreConfig #90

Merged
merged 5 commits into from
Dec 6, 2021
Merged

feat: setStoreV2 with StoreConfig #90

merged 5 commits into from
Dec 6, 2021

Conversation

exromany
Copy link
Contributor

@exromany exromany commented Dec 3, 2021

Implement setStoreV2 from metaplex-foundation/metaplex#1094

@exromany
Copy link
Contributor Author

exromany commented Dec 3, 2021

{
    "jsonrpc": "2.0",
    "error": {
        "code": -32002,
        "message": "Transaction simulation failed: Error processing Instruction 0: Failed to serialize or deserialize account data: Unknown",
        "data": {
            "accounts": null,
            "err": {
                "InstructionError": [
                    0,
                    {
                        "BorshIoError": "Unknown"
                    }
                ]
            },
            "logs": [
                "Program p1exdMJcjVao65QdewkaZRUnU6VPSXhus9n2GzWfh98 invoke [1]",
                "Program log: Error: BorshIoError",
                "Program p1exdMJcjVao65QdewkaZRUnU6VPSXhus9n2GzWfh98 consumed 3637 of 200000 compute units",
                "Program p1exdMJcjVao65QdewkaZRUnU6VPSXhus9n2GzWfh98 failed: Failed to serialize or deserialize account data: Unknown"
            ]
        }
    },
    "id": "a411e5dc-81a5-4df5-b300-faa9ab729b60"
}

austbot
austbot previously approved these changes Dec 3, 2021
export class StoreConfigData extends Borsh.Data<Args> {
static readonly SCHEMA = this.struct([
['key', 'u8'],
['settingsUri', 'string'],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is an option string iun the rust, is that how your represent it herE?

you need to change to this
{ kind: 'option', type: "string"}

@exromany
Copy link
Contributor Author

exromany commented Dec 3, 2021

{
    "jsonrpc": "2.0",
    "error": {
        "code": -32002,
        "message": "Transaction simulation failed: Error processing Instruction 0: custom program error: 0x3b",
        "data": {
            "accounts": null,
            "err": {
                "InstructionError": [
                    0,
                    {
                        "Custom": 59
                    }
                ]
            },
            "logs": [
                "Program p1exdMJcjVao65QdewkaZRUnU6VPSXhus9n2GzWfh98 invoke [1]",
                "Program log: Instruction: Set Store V2",
                "Program log: Derived key invalid",
                "Program p1exdMJcjVao65QdewkaZRUnU6VPSXhus9n2GzWfh98 consumed 19676 of 200000 compute units",
                "Program p1exdMJcjVao65QdewkaZRUnU6VPSXhus9n2GzWfh98 failed: custom program error: 0x3b"
            ]
        }
    },
    "id": "6dcad5e0-59bd-4cbc-9213-72c316eeec7d"
}

@exromany exromany marked this pull request as ready for review December 3, 2021 17:43
@exromany exromany merged commit 0cea78d into main Dec 6, 2021
@exromany exromany deleted the feat/store-config branch December 6, 2021 12:12
@zaxozhu
Copy link
Contributor

zaxozhu commented Dec 6, 2021

🎉 This PR is included in version 4.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants